-
Notifications
You must be signed in to change notification settings - Fork 586
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Paket.Core to support .NET 8.0 #2776
Conversation
Do we need to mention this in the release notes? I suppose it does not affect anyone except maintainers and contributors |
This does affect if you run Paket restore via Fake. But I'd bundle with it under general .NET8 compatibility updates. The issue is that Paket parses Nuget dependencies to paket.lock file and sometimes different frameworks have different dependencies. So if you are so unfortunate that you have a sub-dependency to a package which says "my dependencies in net8.0 are..." then the Paket v6 raises that "there is no such framework as net8", and the restore fails, even if you didn't actually use net8 yourself. Edit: To be clear, this PR fixes that issue. (The main reason I don't want to update readme is that it just creates more merge-conflicts.) |
@Numpsy what do you think? |
There are quite a few |
I see that Paket 8 also added support for .NET Framework 4.8.1, so maybe this change would be helpful to anyone using that as well? |
This is blocking us from upgrading to |
Let me get back home next Monday and I will figure out how to release |
@xperiandri Just a reminder about this, just in case. 🙂 |
@xperiandri if there is anything I can help with that could speed up the release of the pre-release package I would be willing to give it a try. Also should I open a separate issue for this? |
Description
Updated Paket reference.